Skip to content

chore: send out rabbitmq message for visitors approval and cancellation - #1246

Merged
yoganandaness merged 6 commits into
developfrom
SWAP-5075-visa-access-to-visitors
Nov 4, 2025
Merged

chore: send out rabbitmq message for visitors approval and cancellation#1246
yoganandaness merged 6 commits into
developfrom
SWAP-5075-visa-access-to-visitors

Conversation

@yoganandaness

Copy link
Copy Markdown
Contributor

Description

Sending out rabbit MQ message with event name PROPOSAL_UPDATED, when a visitor is accepted or cancelled.

Motivation and Context

Currently rabbit MQ messages are sent out whenever users like Proposers, Data access users are added. This can be utilised by external systems(Like VISA) to add the users into the systems. Along with the Proposers and Data Access Users, this PR is to add the Visitors as well.

How Has This Been Tested

Manually

Fixes

Changes

In the messageBroker, rabbitMq message is sent out when the Visitor is accepted or cancelled

Depends on

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@yoganandaness
yoganandaness requested a review from a team as a code owner November 1, 2025 13:44
@yoganandaness
yoganandaness requested review from TCMeldrum, William-Edwards-STFC and janosbabik and removed request for a team November 1, 2025 13:44
@yoganandaness
yoganandaness enabled auto-merge (squash) November 1, 2025 13:45

@janosbabik janosbabik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @yoganandaness , some comments from my side.

Comment thread apps/backend/src/eventHandlers/messageBroker.ts Outdated
Comment thread apps/backend/src/datasources/postgres/UserDataSource.ts Outdated
@github-actions

github-actions Bot commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@yoganandaness
yoganandaness force-pushed the SWAP-5075-visa-access-to-visitors branch from 5a3ed95 to 3ddff47 Compare November 4, 2025 12:14
@github-actions

github-actions Bot commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@github-actions
github-actions Bot force-pushed the SWAP-5075-visa-access-to-visitors branch from 3ddff47 to 6eef12a Compare November 4, 2025 14:42
…ficeProject/user-office-core into SWAP-5075-visa-access-to-visitors
@github-actions

github-actions Bot commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@yoganandaness
yoganandaness merged commit 431f6c1 into develop Nov 4, 2025
22 checks passed
@yoganandaness
yoganandaness deleted the SWAP-5075-visa-access-to-visitors branch November 4, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants